Skip to content

Feature: Ctrl+LeftClick Drag panning for non-pan tools#935

Merged
winterz merged 2 commits intoKDAB:masterfrom
mattkdab:feature-pan-modifiers
Jan 23, 2024
Merged

Feature: Ctrl+LeftClick Drag panning for non-pan tools#935
winterz merged 2 commits intoKDAB:masterfrom
mattkdab:feature-pan-modifiers

Conversation

@mattkdab
Copy link
Copy Markdown
Member

Requested feature.

Enable Ctrl+Shift+LeftClick Dragging to pan the remote view when using interaction modes other than ViewInteraction.

Should close #933

Requested feature.

Enable Ctrl+Shift+LeftClick Dragging to pan the remote view
when using interaction modes other than ViewInteraction.

Should close KDAB#933
@mattkdab mattkdab requested a review from winterz January 19, 2024 00:17
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 19, 2024

CLA assistant check
All committers have signed the CLA.

@mattkdab
Copy link
Copy Markdown
Member Author

just noticed #741 is sort of the same as this?

@g-fb
Copy link
Copy Markdown

g-fb commented Jan 21, 2024

Any reason for requiring both Ctrl and Shift to be pressed?

Anyway, thank you for adding this.

@mattkdab
Copy link
Copy Markdown
Member Author

Any reason for requiring both Ctrl and Shift to be pressed?

Anyway, thank you for adding this.

Hm, right I suppose it could just be Ctrl, since this is only when not using the pan tool (Ctrl clicking with pan tool performs a Pick Element). I'll update with that change.

@Waqar144
Copy link
Copy Markdown
Contributor

related #741

@mattkdab mattkdab changed the title Feature: Ctrl+Shift+LeftClickDrag panning for non-pan tools Feature: Ctrl+LeftClick Drag panning for non-pan tools Jan 23, 2024
@mattkdab mattkdab requested a review from Waqar144 January 23, 2024 00:15
@winterz
Copy link
Copy Markdown
Contributor

winterz commented Jan 23, 2024

@mattkdab I'm stumped why the cla-assistant doesn't recognize you. I'll approve this and merge.

@winterz winterz merged commit ea79be4 into KDAB:master Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Quick Scenes: always pan view with <modifier>+click

5 participants